searchDisplay: Remove clearDisplayForProvider

Since the two paths that call this want to keep the actor in two different
states, it makes sense to just call the one function that's the same between
both individually.

https://bugzilla.gnome.org/show_bug.cgi?id=693836

diff --git a/js/ui/searchDisplay.js b/js/ui/searchDisplay.js
index 40d29cc7b5b76613a6f3cbbd0602b4949ee8372b..8dbd00e63f6a9e0f9f7ad806b8b225e3422fe8ff 100644 (file)
--- a/js/ui/searchDisplay.js
+++ b/js/ui/searchDisplay.js
@@ -417,12 +417,6 @@ const SearchResults = new Lang.Class({
         }
     },
 
-    _clearDisplayForProvider: function(provider) {
-        let meta = this._metaForProvider(provider);
-        meta.resultDisplay.clear();
-        meta.actor.hide();
-    },
-
     reset: function() {
         this._searchSystem.reset();
         this._statusBin.hide();
@@ -489,7 +483,8 @@ const SearchResults = new Lang.Class({
         let meta = this._metaForProvider(provider);
 
         if (providerResults.length == 0) {
-            this._clearDisplayForProvider(provider);
+            meta.resultDisplay.clear();
+            meta.actor.hide();
             meta.resultDisplay.setResults([], []);
             this._maybeSetInitialSelection();
             this._updateStatusText();
@@ -503,7 +498,7 @@ const SearchResults = new Lang.Class({
                     provider.canLaunchSearch;
 
             provider.getResultMetas(results, Lang.bind(this, function(metas) {
-                this._clearDisplayForProvider(provider);
+                meta.resultDisplay.clear();
                 meta.actor.show();
 
                 // Hiding drops the key focus if we have it